From 01d372422decf27834bf1729b600b1be0e20b97b Mon Sep 17 00:00:00 2001 From: Tor Lillqvist Date: Thu, 20 Mar 2008 03:21:41 +0000 Subject: [PATCH] Don't pass NULL for %s to g_message(). (main): Drop unnecessary call to 2008-03-20 Tor Lillqvist * tests/testfilechooserbutton.c (chooser_update_preview_cb): Don't pass NULL for %s to g_message(). (main): Drop unnecessary call to gtk_widget_destroy(). svn path=/trunk/; revision=19908 --- ChangeLog | 6 ++++++ tests/testfilechooserbutton.c | 11 ++++++----- 2 files changed, 12 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 8ee0c5c110..855c6c1360 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2008-03-20 Tor Lillqvist + + * tests/testfilechooserbutton.c (chooser_update_preview_cb): Don't + pass NULL for %s to g_message(). + (main): Drop unnecessary call to gtk_widget_destroy(). + 2008-03-20 Tor Lillqvist Bug 314084 - GTK+ dialogs should not be placed partially offscreen diff --git a/tests/testfilechooserbutton.c b/tests/testfilechooserbutton.c index 11d7fea78c..f1c1ed3360 100644 --- a/tests/testfilechooserbutton.c +++ b/tests/testfilechooserbutton.c @@ -237,9 +237,12 @@ chooser_update_preview_cb (GtkFileChooser *chooser, gchar *filename; filename = gtk_file_chooser_get_preview_uri (chooser); - g_message ("%s::update-preview\n\tPreview Filename: `%s'\nDone.\n", - G_OBJECT_TYPE_NAME (chooser), filename); - g_free (filename); + if (filename != NULL) + { + g_message ("%s::update-preview\n\tPreview Filename: `%s'\nDone.\n", + G_OBJECT_TYPE_NAME (chooser), filename); + g_free (filename); + } } @@ -358,7 +361,5 @@ main (int argc, gtk_main (); - gtk_widget_destroy (win); - return 0; } -- 2.30.2